Skip to content

add patches to allow offline build of Amber - #23747

Open
branfosj wants to merge 4 commits into
developfrom
amber
Open

add patches to allow offline build of Amber#23747
branfosj wants to merge 4 commits into
developfrom
amber

Conversation

@branfosj

@branfosj branfosj commented Aug 27, 2025

Copy link
Copy Markdown
Member

@branfosj

Copy link
Copy Markdown
Member Author

Test report by @branfosj
Using easyblocks from PR(s) easybuilders/easybuild-easyblocks#3901
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
bear-pg0210u03a - Linux RHEL 8.10, x86_64, Intel(R) Xeon(R) Platinum 8480CL (sapphirerapids), Python 3.6.8
See https://gist.github.com/branfosj/3f774c16a4ba568000335b341219d226 for a full test report.

@boegel

boegel commented Aug 27, 2025

Copy link
Copy Markdown
Member

@branfosj Can you add a README file (symlinked in AmberTools) that explain where these patch files come from, how they were obtained/renamed, and why we're mirroring them?

@branfosj

Copy link
Copy Markdown
Member Author

@branfosj Can you add a README file (symlinked in AmberTools) that explain where these patch files come from, how they were obtained/renamed, and why we're mirroring them?

Done

@boegel

boegel commented Jan 28, 2026

Copy link
Copy Markdown
Member

@branfosj Shouldn't we also update the AmberTools easyconfigs here?

@branfosj

Copy link
Copy Markdown
Member Author

@branfosj Shouldn't we also update the AmberTools easyconfigs here?

If we've decided we want this change in all Amber and AmberTools easyconfigs then I can add PRs for them.

This PR itself was to prove that the easyblock change worked.

@boegel

boegel commented Jan 28, 2026

Copy link
Copy Markdown
Member

@branfosj Have we verified the license aspect of this? Can we actually re-distribute those patch files, especially for Amber?

@branfosj

Copy link
Copy Markdown
Member Author

@branfosj Have we verified the license aspect of this? Can we actually re-distribute those patch files, especially for Amber?

https://github.com/easybuilders/easybuild-easyconfigs/pull/23747/changes#diff-d9acc0beadb617148cc9002ca21a93e6785dc1f5c5de45bfdd42db2c304cdfc2R26

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants